Skip to content

Added log4j with info logs for pulsar-broker unit tests - #37

Merged
merlimat merged 1 commit into
apache:masterfrom
merlimat:master
Sep 27, 2016
Merged

Added log4j with info logs for pulsar-broker unit tests#37
merlimat merged 1 commit into
apache:masterfrom
merlimat:master

Conversation

@merlimat

@merlimat merlimat commented Sep 23, 2016

Copy link
Copy Markdown
Contributor

Motivation

Add default log4j config for pulsar-broker unit tests

@merlimat merlimat added this to the 1.15 milestone Sep 23, 2016
@merlimat merlimat self-assigned this Sep 23, 2016
@yahoocla

yahoocla commented Sep 23, 2016

Copy link
Copy Markdown

CLA is valid!

2 similar comments
@yahoocla

yahoocla commented Sep 23, 2016

Copy link
Copy Markdown

CLA is valid!

@yahoocla

yahoocla commented Sep 23, 2016

Copy link
Copy Markdown

CLA is valid!

rdhabalia
rdhabalia approved these changes Sep 24, 2016
View reviewed changes

@rdhabalia rdhabalia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@merlimat

merlimat commented Sep 26, 2016

Copy link
Copy Markdown
Contributor Author

Apparently, Travis has a limit of 4MB for job output and with this change that would be exceeded. I'll add the log4j conf file with default to off for now.

@merlimat
merlimat merged commit 728091e into apache:master Sep 27, 2016
sijie pushed a commit to sijie/pulsar that referenced this pull request Mar 4, 2018
@srkukarni @sijie
Asyncack (apache#37)
7fbe342 
* Create pulsar-functions module (#1)

* Create pulsar-functions module

* rename `sdk` package to `api`

* Added the first cut of the Java interface for Pulsar functions (#2)

* Made ack async
massakam pushed a commit to massakam/pulsar that referenced this pull request Aug 6, 2020
@hrsakai
Merge pull request apache#37 from freeart/master
1869d2f 
added serialize/deserialize methods to set custom startMessageId in createReader
hrsakai pushed a commit to hrsakai/pulsar that referenced this pull request Dec 10, 2020
@wolfstudy
Merge pull request apache#37 from wolfstudy/xiaolong/fix-zlib-compress
987d2f0 
[issue:34] Fix zlib compress Decompress logic
@xiaotongwang1 xiaotongwang1 mentioned this pull request Aug 4, 2021
Closed
@ZHr-UChiHa ZHr-UChiHa mentioned this pull request Jun 3, 2023
Open
2 tasks
nodece referenced this pull request in nodece/pulsar Mar 12, 2025
@nodece
[feat][broker] Add metrics for marker and snapshot (#37)
63dd4e9 
Signed-off-by: Zixuan Liu <nodeces@gmail.com>
nodece referenced this pull request in nodece/pulsar Mar 27, 2025
@nodece
[feat][broker] Add metrics for marker and snapshot (#37)
ea99d79 
Signed-off-by: Zixuan Liu <nodeces@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rdhabalia rdhabalia rdhabalia approved these changes

Assignees

@merlimat merlimat

Labels

area/test

Projects

None yet

Milestone

1.15

Development

Successfully merging this pull request may close these issues.

3 participants

@merlimat @yahoocla @rdhabalia